Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ci-operator config and job for cri-o 1.14 #4030

Merged
merged 1 commit into from
Jun 26, 2019
Merged

Add ci-operator config and job for cri-o 1.14 #4030

merged 1 commit into from
Jun 26, 2019

Conversation

umohnani8
Copy link
Contributor

Add yaml files to config and jobs for cri-o release-1.14
branch so that we run e2e-aws tests on cri-o PRs.

Signed-off-by: Urvashi Mohnani [email protected]

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 11, 2019
@umohnani8
Copy link
Contributor Author

/test pj-rehearse

2 similar comments
@umohnani8
Copy link
Contributor Author

/test pj-rehearse

@umohnani8
Copy link
Contributor Author

/test pj-rehearse

@umohnani8
Copy link
Contributor Author

@petr-muller the e2e-aws and launch-aws tests are failing to start up a cluster. This has been happening since yesterday. Am I messing something up or is this just test flakes?
The config and job match what we have for cri-o 1.13 in #3962 and those tests are passing.

@petr-muller
Copy link
Member

/test pj-rehearse

@umohnani8 I don't know. The rehearsals execute the added/changed jobs as if they were run against the target branch HEAD, so in the case of e2e tests their results depend on how healthy openshift currently is. It's certainly known to flake a bit (current build cop usually knows if there's currently something going on). The rehearsals do not block the merge, so if it looks like the jobs work fine and the tests inside flake them, you can merge them.

@openshift-merge-robot
Copy link
Contributor

/test core-valid
/test core-dry

@umohnani8
Copy link
Contributor Author

@petr-muller got it, thanks! Trying to debug the problem here, seems something is off with the cri-o being built for the ci.

@umohnani8
Copy link
Contributor Author

/test pj-rehearse

@cgwalters
Copy link
Member

cgwalters commented Jun 25, 2019

Look at the log bundle and drill down into one of the master's journals:

Jun 19 13:13:46 ip-10-0-129-84 hyperkube[944]: F0619 13:13:46.174808     944 server.go:264] failed to run Kubelet: failed to create kubelet: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial unix /var/run/crio/crio.sock: connect: no such file or directory"

And that's because:

-- Logs begin at Wed 2019-06-19 13:09:22 UTC, end at Wed 2019-06-19 13:41:43 UTC. --
Jun 19 13:13:43 ip-10-0-129-84 systemd[1]: Starting Open Container Initiative Daemon...
Jun 19 13:13:44 ip-10-0-129-84 crio[928]: time="2019-06-19 13:13:44.541376996Z" level=fatal msg="config validation: invalid plugin_dirs entry: stat /var/lib/cni/bin: no such file or directory"
Jun 19 13:13:44 ip-10-0-129-84 systemd[1]: crio.service: Main process exited, code=exited, status=1/FAILURE
Jun 19 13:13:44 ip-10-0-129-84 systemd[1]: crio.service: Failed with result 'exit-code'.

@umohnani8
Copy link
Contributor Author

/test pj-rehearse

1 similar comment
@umohnani8
Copy link
Contributor Author

/test pj-rehearse

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 25, 2019

@umohnani8: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/rehearse/cri-o/cri-o/release-1.14/unit 92224b29253ce0fd111b2964f18a4d1589e03513 link /test pj-rehearse

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Add yaml files to config and jobs for cri-o release-1.14
branch so that we run e2e-aws tests on cri-o PRs.

Signed-off-by: Urvashi Mohnani <[email protected]>
@mrunalp
Copy link
Member

mrunalp commented Jun 25, 2019

@smarterclayton @stevekuznetsov ptal. This is ready.

@stevekuznetsov
Copy link
Contributor

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 26, 2019
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: stevekuznetsov, umohnani8

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 26, 2019
@openshift-merge-robot openshift-merge-robot merged commit 872b1e0 into openshift:master Jun 26, 2019
@openshift-ci-robot
Copy link
Contributor

@umohnani8: Updated the following 3 configmaps:

  • job-config-misc configmap in namespace ci using the following files:
    • key cri-o-cri-o-release-1.14-presubmits.yaml using file ci-operator/jobs/cri-o/cri-o/cri-o-cri-o-release-1.14-presubmits.yaml
  • ci-operator-misc-configs configmap in namespace ci using the following files:
    • key cri-o-cri-o-release-1.14.yaml using file ci-operator/config/cri-o/cri-o/cri-o-cri-o-release-1.14.yaml
  • ci-operator-misc-configs configmap in namespace ci-stg using the following files:
    • key cri-o-cri-o-release-1.14.yaml using file ci-operator/config/cri-o/cri-o/cri-o-cri-o-release-1.14.yaml

In response to this:

Add yaml files to config and jobs for cri-o release-1.14
branch so that we run e2e-aws tests on cri-o PRs.

Signed-off-by: Urvashi Mohnani [email protected]

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants